Skip to content

Conversation

@Samoed
Copy link
Member

@Samoed Samoed commented Feb 20, 2025

No description provided.

@Samoed Samoed requested a review from voorhs February 20, 2025 20:18
@Samoed
Copy link
Member Author

Samoed commented Feb 20, 2025

Локально тесты проходят, почему ломаются не понимаю

@Samoed
Copy link
Member Author

Samoed commented Feb 21, 2025

Возможно можно убрать просто кониг searchspace и использовать один этот

Comment on lines +184 to +199
class TaskConfig(BaseModel):
"""Configuration for the task to optimize."""

search_space: OptimizationSearchSpaceConfig
"""Path to the search space configuration file. If None, the default search space will be used"""
sampler: SamplerType = "brute"


class OptimizationConfig(BaseModel):
"""Configuration for the optimization process."""

data_config: DataConfig = DataConfig()
task_config: TaskConfig
logging_config: LoggingConfig = LoggingConfig()
vector_index_config: VectorIndexConfig = VectorIndexConfig()
seed: PositiveInt = 42
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

это конфиги и они должны быть в autointent.configs

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Могу сюда все конфиги тогда перенести

@voorhs voorhs merged commit 54b8fad into dev Feb 24, 2025
22 checks passed
@voorhs voorhs deleted the add_full_config_validation branch February 24, 2025 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants